Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade H2 to 2.1.210 #24663

Merged
merged 1 commit into from
Apr 6, 2022
Merged

Upgrade H2 to 2.1.210 #24663

merged 1 commit into from
Apr 6, 2022

Conversation

gsmet
Copy link
Member

@gsmet gsmet commented Mar 31, 2022

Fixes #9354

@gsmet gsmet requested a review from Sanne March 31, 2022 10:08
@quarkus-bot quarkus-bot bot added the area/dependencies Pull requests that update a dependency file label Mar 31, 2022
@gsmet gsmet mentioned this pull request Mar 31, 2022
@quarkus-bot

This comment has been minimized.

@gsmet
Copy link
Member Author

gsmet commented Mar 31, 2022

Ah, looks like it will require some more fun.

@gsmet
Copy link
Member Author

gsmet commented Mar 31, 2022

It should be much better now.

@Sanne
Copy link
Member

Sanne commented Mar 31, 2022

I see some local tests are failing, as it's generating this statement:
insert into User (name, id) values (?, ?)

User is now a keyword in H2, so the tests should be adapted in the model: use @Table to have it generate a different table name.

@gsmet
Copy link
Member Author

gsmet commented Mar 31, 2022

@Sanne yeah I'll wait for the full report. I have run the jpa-h2 one in native but apparently some others are complaining. I will adjust accordingly.

@quarkus-bot

This comment has been minimized.

@quarkus-bot quarkus-bot bot added area/flyway area/hibernate-orm Hibernate ORM area/liquibase area/persistence OBSOLETE, DO NOT USE area/spring Issues relating to the Spring integration labels Mar 31, 2022
@quarkus-bot

This comment has been minimized.

@gsmet
Copy link
Member Author

gsmet commented Apr 1, 2022

@Sanne it's now ready. The rest-client issue is totally unrelated. Quite an adventure.

@Sanne
Copy link
Member

Sanne commented Apr 1, 2022

@gsmet I see some failures in module quarkus-integration-test-hibernate-validator as well - are you sure they are unrelated?

@Sanne
Copy link
Member

Sanne commented Apr 1, 2022

e.g. io.quarkus.it.hibernate.validator.HibernateValidatorFunctionalityTest is failing and appears to also use ORM + H2:

javax.validation.ConstraintViolationException: Validation failed for classes [io.quarkus.it.hibernate.validator.orm.TestEntity] during persist time for groups [javax.validation.groups.Default, ]
List of constraint violations:[
	ConstraintViolationImpl{interpolatedMessage='must not be null', propertyPath=validatedField, rootBeanClass=class io.quarkus.it.hibernate.validator.orm.TestEntity, messageTemplate='{javax.validation.constraints.NotNull.message}'}
]
	at org.hibernate.cfg.beanvalidation.BeanValidationEventListener.validate(BeanValidationEventListener.java:140)
	at org.hibernate.cfg.beanvalidation.BeanValidationEventListener.onPreInsert(BeanValidationEventListener.java:80)
	at org.hibernate.action.internal.EntityInsertAction.preInsert(EntityInsertAction.java:214)
	at org.hibernate.action.internal.EntityInsertAction.execute(EntityInsertAction.java:100)
	at org.hibernate.engine.spi.ActionQueue.executeActions(ActionQueue.java:604)

@gsmet
Copy link
Member Author

gsmet commented Apr 1, 2022

@Sanne do you see test failures? Because I ran the whole extensions test suite and then the whole integration-tests suite locally with my latest changes in this PR.
And CI didn't report any failure for Validator either.

@gsmet
Copy link
Member Author

gsmet commented Apr 1, 2022

BTW what you pasted is normal, it's just HV being triggered.

@Sanne
Copy link
Member

Sanne commented Apr 1, 2022

yes I had local test failures in the validator extension - I believe I found a match on the CI logs but I might be wrong about CI.

FWIW this is how I run test locally:

mvn clean install -pl :quarkus-hibernate-orm -amd

I just double checked and don't have the same failures when running without your PR

@gsmet
Copy link
Member Author

gsmet commented Apr 1, 2022

Are you sure you are testing the latest changes? Because I just rebuilt everything in this PR and tested integration-tests/hibernate-validator and it's green. And CI agrees with me.

I can rebase and force push to get the rest client fix.

@gsmet
Copy link
Member Author

gsmet commented Apr 1, 2022

I force pushed a rebase.

@Sanne
Copy link
Member

Sanne commented Apr 1, 2022

I'll run it again...

@gsmet
Copy link
Member Author

gsmet commented Apr 1, 2022

CI looks happy.

@Sanne
Copy link
Member

Sanne commented Apr 5, 2022

sorry for not getting back to you yet :/ Having some issues with my environments(s): since I can't seem to get consistent build results locally even on "main" it's hard to say definitely how the upgrade is breaking things.

I'll try to cap this to today as I don't want to hold you for too long, but I have a strong impression that I have more issues when the H2 version is updated.

@gsmet
Copy link
Member Author

gsmet commented Apr 5, 2022

My guess is that you somehow have something inconsistent or you're not building everything that should be built.

@Sanne
Copy link
Member

Sanne commented Apr 6, 2022

it's finally working fine 👍
sorry for the delay

@Sanne Sanne merged commit 5296576 into quarkusio:main Apr 6, 2022
@quarkus-bot quarkus-bot bot added this to the 2.9 - main milestone Apr 6, 2022
@quarkus-bot quarkus-bot bot added the kind/enhancement New feature or request label Apr 6, 2022
@gsmet
Copy link
Member Author

gsmet commented Apr 6, 2022

Woot, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/dependencies Pull requests that update a dependency file area/flyway area/hibernate-orm Hibernate ORM area/liquibase area/persistence OBSOLETE, DO NOT USE area/spring Issues relating to the Spring integration area/testing kind/enhancement New feature or request release/breaking-change release/noteworthy-feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update h2 database
2 participants